Support JDK17#660

Merged

xerial merged 4 commits intodevelopfrom

Jun 28, 2022

Member

xerial commented Jun 28, 2022 •

edited

Loading

To use JDK17 + DirectByteBuffer, we need to set two JVM options:

--add-opens=java.base/java.nio=ALL-UNNAMED
--add-opens=java.base/sun.nio.ch=ALL-UNNAMED

This PR adds:

CI for JDK17
Add a workaround even if these JVM options are not set.
Add a warning message when DirectByteBuffer is used in JDK17 without setting these JVM options

Related approaches:

[SPARK-36704][CORE] Expand exception handling to more Java 9 cases where reflection is limited at runtime, when reflecting to manage DirectByteBuffer settings apache/spark#33947
[SPARK-36796][BUILD][CORE][SQL] Pass all sql/core and dependent modules UTs with JDK 17 except one case in postgreSQL/text.sql apache/spark#34153 (--add-open JVM options to use reflection access)
Use DirectBuffer interface to get buffer address airlift/slice#130

Fixes #600

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet